Передать параметр ajax-запросу

[napilnik]

Есть код:

for (i=index; i>=1; i--){
  name = mainAdress + 'data/news/news' + i + '.xml';
  $.get(name, {}, function(xml){
    $(xml).find('document').each(function(){
      $('#n' + i + ' h3').text($(this).find('title').text());
      $('#n' + i + ' small').text($(this).find('date').text());
      $('#n' + i + ' p').text($(this).find('text').text());
    });
  });
}

Но в самом запросе переменная i не видна. Как ее туда передать?

[razerxxx]

попробуй сделать так:

var sendToServer = function(i) {
  $.get(name, {}, function(xml){
    $(xml).find('document').each(function(){
      $('#n' + i + ' h3').text($(this).find('title').text());
      $('#n' + i + ' small').text($(this).find('date').text());
      $('#n' + i + ' p').text($(this).find('text').text());
    });
  });
}
for (i=index; i>=1; i--){
  sendToServer(i);
}

[Gvozd]

http://javascript.ru/basic/closure#p...-ispolzovaniya

[napilnik]

спасибо, помогло. но страница стала намного дольше грузится( нельзя это как-то оптимизировать?

[Gvozd]

napilnik,
код который вам дал razerxxx - неправильный.
Она забыл перенести одну строчку из вашего кода

var sendToServer = function(i) {
  var name = mainAdress + 'data/news/news' + i + '.xml';
  $.get(name, {}, function(xml){
    $(xml).find('document').each(function(){
      $('#n' + i + ' h3').text($(this).find('title').text());
      $('#n' + i + ' small').text($(this).find('date').text());
      $('#n' + i + ' p').text($(this).find('text').text());
    });
  });
}
for (i=index; i>=1; i--){
  sendToServer(i);
}

PS не рекомендуется использовать переменную с названием name, так как уже есть такая глобальная переменная.
В вашем первом коде вы похоже как раз и объявили ее как глобальную, а это чревато проблемами, например в Chrome

[napilnik]

а где можно прочитать, какие имена уже заняты?

[Gvozd]

а где можно прочитать, какие имена уже заняты?

http://javascript.ru/ecma/part7#a-7.5.1
Это те, кот оыре в принципе нельзя использовать.
С name же ситуация несколько иная - это свойство объекта window, а значит и глобальная переменная в браузерах, при чем в Chrome она всегда приводится к строке
И чтобы таких проблем не возникало, надо просто не использовать глобальных переменных без крайней необходимости на то, и оборачивать свой код в замыкания, объявляя переменные как локальные для замыкания

[razerxxx]

Ну а вообще ко всему что уже написано - делать аякс запрос в цикле - как-то не очень, хорошо. Т.е работать то оно работает, но внутреннее чувство подсказывает мне что так делать не надо, лучше попробовать как-то это переделать

[Gvozd]

делать аякс запрос в цикле - как-то не очень, хорошо.

чем именно?

Т.е работать то оно работает, но внутреннее чувство подсказывает мне что так делать не надо

когда кажется - крестится надо

[tadjik1]

да нет, всё гуд — запросы асинхронные, следующий запускается, не дожидаясь пока выполнится предыдущий.
не волнуйтесь